home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part1 / 8638 < prev    next >
Encoding:
Text File  |  1996-08-05  |  2.6 KB  |  92 lines
  1. Path: news.rain.org!usenet
  2. From: "Guus Leeuw jr." <guusl@eiffel.com>
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: What is wrong with the code please?
  5. Date: Sun, 25 Feb 1996 10:47:59 -0800
  6. Organization: Interactive Software Engineering Inc. http://www.eiffel.com/
  7. Message-ID: <3130AEDF.3FFCCCBE@eiffel.com>
  8. References: <4ghbip$a68@nuke.csu.net>
  9. NNTP-Posting-Host: @outback.eiffel.com
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.0 (X11; I; Linux 1.2.8 i586)
  14.  
  15. Syrous Etezadi Amoli wrote:
  16. > 
  17. > Hi
  18. > 
  19. > Please tell me what is wrong with the following code which implements a
  20. > linked list in BC++ 4.5 with TListImp?
  21. > 
  22. > #include <classlib\listimp.h>
  23. > #include <iostream.h>
  24. > #include <String.h>
  25. > 
  26. > void main()
  27. > {
  28. > 
  29. > class CIndexListElement : public: string{
  30. >  private:
  31. >   string subject;
  32. >   long start;
  33. >   long end;
  34. >  public:
  35. > 
  36. >   DWORD GetStart(void) {return start;};
  37. >   DWORD GetEnd(void) { return end;};
  38. >   string GetSubject(void) {return subject;};
  39. >   void SetStart(DWORD x) {start = x;};
  40. >   void SetEnd(DWORD x) {end = x;};
  41. >   void SetSubject(string s) { subject = s;};
  42. >   private:
  43. > 
  44. > };
  45. > TListImp<CIndexListElement> m_SubjectList;
  46. > 
  47. > CIndexListElement *new_entry=  new CIndexListElement;
  48. > new_entry->SetSubject("1st subject");
  49. > new_entry->SetEnd(1);
  50. > new_entry->SetStart(1);
  51. > m_SubjectList.Add(new_entry);
  52.  
  53. I bet this call to `Add' fails to compile, huh?
  54.  
  55. Let's see... new_entry is a pointer to an object of type CIndexListElement;
  56. m_SubjectList is an object of type TListImp<CIndexListElement>. Hmm, your
  57. template is instantiated with type CIndexListElement and you're trying to put
  58. pointers in it. Either, dereference your pointer, or instantiate your
  59. template to hold CIndexListElement*.
  60.  
  61. > new_entry->SetSubject("2ND subject");
  62. > new_entry->SetEnd(2);
  63. > new_entry->SetStart(2);
  64. > m_SubjectList.Add(new_entry);
  65. > new_entry->SetSubject("3RD subject");
  66. > new_entry->SetEnd(3);
  67. > new_entry->SetStart(3);
  68. > m_SubjectList.Add(new_entry);
  69. > 
  70.  
  71. When you declare m_SubjectList to be a TListImp<CIndexListElement*>, you
  72. gotta be darn sure, that the list copies the objects put in it (i.e. does Add
  73. clone the argument?). If not, you might want to get new objects of type
  74. CIndexListElement for every Add.
  75.  
  76. > TListIteratorImp<cIndexListElement> next(m_SubjectList);
  77. > cout << "\n The stored items are:\n" << next++;
  78. > 
  79. > while (next)
  80. >   cout << ", " << next++;
  81. > cout << endl;
  82.  
  83. Here is one (maybe two) other problem:
  84. a) you don't delete new_entry;
  85. (b) your program doesn't return a result. Plus that main is declared to
  86. return a void type...)
  87. > }
  88. > 
  89.  
  90. Hope this helps,
  91.     Guus
  92.